Skip to content

update caesar checks for new requirements#76

Open
curiouskiwi wants to merge 1 commit intocs50:masterfrom
curiouskiwi:patch-1
Open

update caesar checks for new requirements#76
curiouskiwi wants to merge 1 commit intocs50:masterfrom
curiouskiwi:patch-1

Conversation

@curiouskiwi
Copy link
Contributor

Add two more tests for caesar per the changes to the spec for 2018, namely

  • if the user provides no command-line arguments, or two or more, it prints Usage: ./caesar key.
  • checks to make sure that each character of that command line argument is a decimal digit (i.e., 0, 1, 2, etc.) and, if any of them are not, terminates (with a return code of 1) after printing the message Usage: ./caesar key.

Add two more tests for caesar per the changes to the spec for 2018, namely

* if the user provides no command-line arguments, or two or more, it prints Usage: ./caesar key. 
* checks to make sure that each character of that command line argument is a decimal digit (i.e., 0, 1, 2, etc.) and, if any of them are not, terminates (with a return code of 1) after printing the message Usage: ./caesar key.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant